Add neutronPowerOn/neutronPowerOff hooks to NeutronBackend#19845
Add neutronPowerOn/neutronPowerOff hooks to NeutronBackend#19845JakeStevens wants to merge 1 commit into
Conversation
Summary: To minimize power, the NPU should be turned on and off, per: https://docs.nxp.com/bundle/AN14700/page/topics/imx_rt700_system_details.html This relies on SDK code. Some users may wrap this code for protection. So, we want two things: 1) (potentially) turnOn -> execute NPU -> turnOff 2) ability to override turnOn/Off implementations Further, some may want to leave these stubs so that the ET backend does not control it for them. Differential Revision: D106668625
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19845
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 1 New Failure, 2 Unrelated FailuresAs of commit b316aee with merge base acce7cd ( NEW FAILURE - The following job has failed:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@JakeStevens has exported this pull request. If you are a Meta employee, you can view the originating Diff in D106668625. |
This PR needs a
|
|
I see there are neutronSuspend and neutronResume functions in the neutron header. Maybe can just use those instead? |
Summary:
To minimize power, the NPU should be turned on and off, per:
https://docs.nxp.com/bundle/AN14700/page/topics/imx_rt700_system_details.html
This relies on SDK code.
Some users may wrap this code for protection.
So, we want two things:
Further, some may want to leave these stubs so that the ET backend does not control it for them.
Differential Revision: D106668625
cc @robert-kalmar @digantdesai @rascani